Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: create a private package for shared eslint configs #11240

Merged
merged 10 commits into from
Jan 13, 2025

Conversation

benelan
Copy link
Member

@benelan benelan commented Jan 9, 2025

Summary

  • Create a new package in packages/eslint-config-calcite that contains shared ESLint configurations for importing in other packages.
  • Setup the components and design-tokens packages to use the new shared ESLint configs.
  • Add typescript-eslint coverage to our github scripts
  • Check css files in lint-staged (previously it was only scss)
  • Disable no-unexpected-newline, which conflicts with prettier
  • Various other cleanup, such as formatting json files in .vscode/ and removing cspell ignore comments in favor of dictionary additions..
  • Run npm run lint so unrelated changes don't appear in subsequent PRs

@github-actions github-actions bot added the chore Issues with changes that don't modify src or test files. label Jan 9, 2025
@jcfranco jcfranco added the skip visual snapshots Pull requests that do not need visual regression testing. label Jan 9, 2025
@benelan benelan requested a review from alisonailea as a code owner January 9, 2025 22:33
@benelan benelan changed the title ci: improve linting of github scripts and css build: create a private package for shared eslint configs Jan 10, 2025
@benelan benelan requested a review from jcfranco January 11, 2025 00:00
@benelan
Copy link
Member Author

benelan commented Jan 11, 2025

@jcfranco can you take another look at this? I significantly reworked the PR since your approval. @alisonailea can you take a quick look at the design tokens package changes as well?

Copy link
Member

@jcfranco jcfranco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! 🚀

@@ -14,6 +14,7 @@ function deeplyNestedColorValue(val) {

if (val.includes("rgba") || color.getAlpha() === 0) {
const hexMatch = val.match(/#[\w\d]{3,6}/g);
// eslint-disable-next-line no-useless-escape
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add an explainer to this disabled line? Applies to other disabled rules.

…mprovements

* origin/dev:
  fix(input): update files property on input event when type="file" (#11262)
@benelan benelan merged commit 03a0aad into dev Jan 13, 2025
12 checks passed
@benelan benelan deleted the benelan/more-linting-improvements branch January 13, 2025 06:47
benelan added a commit that referenced this pull request Jan 13, 2025
* dev:
  build: create a private package for shared eslint configs (#11240)
  build: simplify version placeholder replacement (#11266)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Issues with changes that don't modify src or test files. skip visual snapshots Pull requests that do not need visual regression testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants